sl rn saved queries w no exports #6798
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
github-actions
bot
added
content
| ## January 2025 | ||
|
|
||
| - **Fix**: Fixed an issue where [saved queries](/docs/build/saved-queries) with no [exports](/docs/build/saved-queries#configure-exports) would fail with an `UnboundLocalError`. Previously, attempting to process a saved query without any exports would cause an error due to an undefined relation variable. Exports are optional and this fix ensures saved queries without exports don't fail. |
There was a problem hiding this comment.
| - **Fix**: Fixed an issue where [saved queries](/docs/build/saved-queries) with no [exports](/docs/build/saved-queries#configure-exports) would fail with an `UnboundLocalError`. Previously, attempting to process a saved query without any exports would cause an error due to an undefined relation variable. Exports are optional and this fix ensures saved queries without exports don't fail. | |
| - **Fix**: Fixed an issue where [saved queries](/docs/build/saved-queries) with no [exports](/docs/build/saved-queries#configure-exports) would fail with an `UnboundLocalError`. Previously, attempting to process a saved query without any exports would cause an error, due to an undefined relation variable. Exports are optional and this fix ensures saved queries without exports, don't fail. |
There was a problem hiding this comment.
thanks @nataliefiann ! not sure if we need the commas here and i wonder if it actually disrupts the flow. the clause "don't fail" is directly linked to the preceding phrase so it looks like the comma interrupts it. how about adding a comma after the 'Exports are optional, and this...:
There was a problem hiding this comment.
going to commit it with the: 'Exports are optional, and this...' but please let me know if you have any strong feelings about this version! thank yoU!
There was a problem hiding this comment.
Hiya @mirnawong1
Thanks for creating this! I've approved this for you and left 2 non-block suggestions (jut 2 commas to break up the sentence).
Kind Regards
Natalie
this pr adds a SL release note entry that adds a fix made to ensure saved queries w no exports don't fail
linear: https://linear.app/dbt-labs/issue/SL-3174/add-support-for-multiple-time-spines-when-using-deferral-in-mantle
🚀 Deployment available! Here are the direct links to the updated files: